Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export directive to support batch processing. #99

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

boneill42
Copy link

@boneill42 boneill42 commented Jan 4, 2017

Similar to:
graphql/graphql-js#462

This PR is an initial stab at an export directive. It exposes variable_values in the context, and builds up a list of values for the variable during processing of the query.

@boneill42
Copy link
Author

Any thoughts on this? It is worth resolving the conflicts and getting it sync'd up with master?

@gsjurseth
Copy link

I would love to see this feature... I'm a bit new to graphql, admittedly, but was a little surprised that this isn't part of the core feature set. I did see some discussions about adding to apollo instead, but as I do think it's better suited in core...

Just my anonymous $.02.

@eamigo86
Copy link

eamigo86 commented Dec 8, 2017

Hello @boneill42, some progress in this feature??

@boneill42
Copy link
Author

Sorry, I haven't made any progress beyond what you see here.

@syrusakbary syrusakbary force-pushed the master branch 3 times, most recently from b1f26c1 to a7ce75e Compare July 19, 2018 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants